Skip to content

feat: migrate to vue3#2279

Draft
mejo- wants to merge 21 commits intomainfrom
feat/vue3
Draft

feat: migrate to vue3#2279
mejo- wants to merge 21 commits intomainfrom
feat/vue3

Conversation

@mejo-
Copy link
Member

@mejo- mejo- commented Feb 18, 2026 •
edited
Loading

📝 Summary

Resolves: #1399

🚧 TODO

  • Subpages don't get rendered.
  • PageInfoBar component is not rendered in reader, probably because it's a Vue3 component now.
  • Files app header is not displayed.
  • Test drag and drop in page list intensively as we migrated to a new wrapper library. revertOnSpill is not supported by it.
  • Test opening anchor links with headings. vue-router uses history for hash links in v4/v5, no idea if this is a problem.

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests
  • Documentation (README or documentation) has been updated or is not required

@mejo- mejo- self-assigned this Feb 18, 2026
@mejo- mejo- mentioned this pull request Feb 24, 2026
Closed
5 tasks
@mejo- mejo- force-pushed the feat/vue3 branch 4 times, most recently from e410bf9 to a5cffae Compare February 24, 2026 16:57
mejo- added 18 commits February 25, 2026 15:21
@mejo-
feat: upgrade dependencies to Vue3
b70cf1c 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(vue): migrate from Vue.extend() to createApp()
12e779d 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(vue): don't use @<event>.native
610396b 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(vue): migrate <var>.sync to v-model:<var>
5d2a302 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(vue): move from beforeDestroy to beforeUnmount
1f5f68d 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(vueRouter): migrate to vue-router 5 syntax
7df8f03 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(vueRouter): use fullPath for comparison to take query params into…
872433a 
… account

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(pinia): don't use Vue.set() and this.$set() and this.$delete()
8751bc6 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(SkeletonLoading): key attribute in v-for component
7084bec 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(sortable): migrate from vuedraggable to vue-draggable-plus
dffcb8e 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
test(playwright): fix typescript type error in create-collective.ts
6fd33ae 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
chore: migrate to vue3 eslint rules
4304e51 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(vue): declare emitted events in components
629b04f 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(PageListItem): call function instead of expression
0cf69c0 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(SharingEntryLink): don't mutate property
114b53a 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(editor): use markRaw for Text editor instances
528c792 
Our vue3 app doesn't like the editor component being vue2. Using
`markRaw()` will prevent vue from converting it to a proxy, which means
it won't be reactive.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(FilesCollectivesHeader): fix updating path and migrate to typescript
996b468 
Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(PageInfoBar): pass to Text as custom web component
9cc8b08 
Required as we now use Vue 3 and Text uses Vue 2.

Another advantage is that we take care of the component properties
within Collectives, no need to update them through Text.

Signed-off-by: Jonas <jonas@freesources.org>
mejo- added 2 commits February 25, 2026 17:26
@mejo-
fix(vue): use beforeUnmount instead of unmounted for garbage coll…
03b2993 
…ection

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(NcActions): rename child components to NcAction*
500de05 
required for them to be rendered with nextcloud-vue v9.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
fix(cypress): in Vue 3, it's app.config.globalProperties.$router
2955d93 
Signed-off-by: Jonas <jonas@freesources.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@max-nextcloud max-nextcloud Awaiting requested review from max-nextcloud max-nextcloud will be requested when the pull request is marked ready for review max-nextcloud is a code owner

@silverkszlo silverkszlo Awaiting requested review from silverkszlo silverkszlo will be requested when the pull request is marked ready for review silverkszlo is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@mejo- mejo-

Labels

2. developing technical debt

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Migrate to Vue3

1 participant

@mejo-